Skip to content

Add Logging to README.md.#1901

Merged
daspecster merged 2 commits intogoogleapis:masterfrom
daspecster:add-logging-to-readme
Jun 27, 2016
Merged

Add Logging to README.md.#1901
daspecster merged 2 commits intogoogleapis:masterfrom
daspecster:add-logging-to-readme

Conversation

@daspecster
Copy link
Contributor

@tseaver LMKWYT and if there's anything I should add.
/cc @dhermes

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jun 24, 2016
@daspecster daspecster changed the title Add Logging to README.md. closes #1898. Add Logging to README.md. Jun 24, 2016
logger = client.logger('log_name')
logger.log_text("A simple entry") # API call


This comment was marked as spam.

@tseaver
Copy link
Contributor

tseaver commented Jun 26, 2016

LGTM, even if you decide to ignore my nitpick.

@tseaver
Copy link
Contributor

tseaver commented Jun 26, 2016

Hmm, maybe flesh the example out to show fetching entries? E.g.:

>>> entries, token = logger.list_entries()
>>> for entry in entries:
...   print entry.payload
A simple entry

@tseaver
Copy link
Contributor

tseaver commented Jun 26, 2016

@daspecster reminder to close #1898 after merge (I think having closes #1898 at the end of the first line of the commit message hides it from Github).

@tseaver
Copy link
Contributor

tseaver commented Jun 27, 2016

LGTM.

@daspecster daspecster merged commit 5c4db0d into googleapis:master Jun 27, 2016
@daspecster daspecster deleted the add-logging-to-readme branch June 27, 2016 16:18
@daspecster
Copy link
Contributor Author

@tseaver looks like the closes #1898 in the commit message worked!

@tseaver tseaver mentioned this pull request Jun 29, 2016
parthea added a commit that referenced this pull request Nov 24, 2025
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
parthea pushed a commit that referenced this pull request Mar 2, 2026
Updated `system_tests/system_tests_sync/test_default.py` and `system_tests/system_tests_async/test_default.py` to catch `RefreshError` and check for "invalid_grant" in the error message when running with `authorized_user.json` credentials. This allows the test to pass by confirming the library correctly received the error from the server, acknowledging that the CI credentials are often stale.

This builds on @daniel-sanche' work in [PR #1883](https://github.com/googleapis/google-auth-library-python/pull/1883).

Recent changes to `main` appear to have adjusted the response values in these tests and was leading to them failing during CI/CD presubmit tests ([see example below](https://github.com/googleapis/google-auth-library-python/pull/1901#issuecomment-3682277336)). This update now prevents those failing tests.


---
*PR created automatically by Jules for task [5370487295365926138](https://jules.google.com/task/5370487295365926138) started by @chalmerlowe*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants